Skip to content

feat(reference-app): collapse the code and token values behind disclosures - #193

Merged
ardetrick merged 3 commits into
mainfrom
feat/reference-app-collapse-content
Jul 26, 2026
Merged

feat(reference-app): collapse the code and token values behind disclosures#193
ardetrick merged 3 commits into
mainfrom
feat/reference-app-collapse-content

Conversation

@ardetrick

@ardetrick ardetrick commented Jul 26, 2026

Copy link
Copy Markdown
Owner

The callback page leads with a prominent Exchange button and an honest note that a real app does this server-side; the raw code and client credentials move into disclosures. The tokens page leads with a success line and tucks the raw response and decoded claims away. This keeps variable-length content out of the default view so every screen is a fixed size.

Stacked on #192 — review/merge after it.

…g pages

Vendors Pico v2.1.1 as static/pico.min.css and restyles the pages as a centered card. Login now pre-fills the demo credentials with a caption so they are obvious. Page titles are normalized to h1, and the screenshot viewport is pinned to 576x520 so captures are a uniform, GitHub-column-friendly size.
…sures

The callback page leads with a prominent Exchange button and an honest note that a real app does this server-side; the raw authorization code and the client credentials move into <details> disclosures. The tokens page leads with a success line and tucks the raw token response and decoded claims behind disclosures. This keeps the variable-length, non-deterministic content out of the default view so every screen is a fixed size.
Base automatically changed from feat/reference-app-pico-styling to main July 26, 2026 04:02
@ardetrick
ardetrick merged commit f05b0b1 into main Jul 26, 2026
1 check passed
@ardetrick
ardetrick deleted the feat/reference-app-collapse-content branch July 26, 2026 04:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant